Skip to content

Conversation

@ghost
Copy link

@ghost ghost commented May 2, 2025

What this PR does / why we need it:
docker

Which issue(s) this PR closes:

Special notes for your reviewer:
If you normally start the Docker image with docker-compose and Docker is configured to restart all running Docker instances upon a system restart, the error occurs. This happens because the script specifies that if a program does not return exit 0, the script should terminate, and as a result, the entire startup process fails.

Suggestions on how to test this:
During the normal start, all three or at least one of the following parameters must be specified: LINUX_PASSWORD, PAYARA_ADMIN_PASSWORD, DOMAIN_PASSWORD.

@poikilotherm
Copy link
Contributor

poikilotherm commented May 2, 2025

Good catch figuring out that the failing passwd command breaks the chain! Thanks for figuring this out!

I would prefer handling the error properly by using an or (||) behind the command, making it better comply with shellscript good practices. What do you think?

Also, we should probably not just fix it in develop, but also add backports for the last three releases in modules/container-base/src/backports

@pdurbin pdurbin moved this to Ready for Triage in IQSS Dataverse Project May 2, 2025
@pdurbin pdurbin assigned ghost May 2, 2025
@pdurbin pdurbin added Size: 3 A percentage of a sprint. 2.1 hours. Component: Containers Anything related to cloudy Dataverse, shipped in containers. labels May 2, 2025
@pdurbin
Copy link
Member

pdurbin commented May 2, 2025

@BenediktMeierUIT thanks for the PR! Please check out Oliver's comment. Also, I added this to next week's agenda for https://ct.gdcc.io. I hope you can make it!

@ghost
Copy link
Author

ghost commented May 2, 2025

I can’t tell you exactly, as today is my last day and I’ll be on parental leave, and I’m not sure if my contract will be extended afterwards.

@poikilotherm
Copy link
Contributor

Good luck with your little one! If you won't make it, I can take it up from here. Thanks again!

@ofahimIQSS ofahimIQSS moved this from Ready for Triage to Ready for Review ⏩ in IQSS Dataverse Project May 13, 2025
@scolapasta scolapasta assigned poikilotherm and unassigned ghost May 14, 2025
@scolapasta scolapasta moved this from Ready for Review ⏩ to Ready for QA ⏩ in IQSS Dataverse Project May 14, 2025
@pdurbin pdurbin moved this from Ready for QA ⏩ to In Review 🔎 in IQSS Dataverse Project May 14, 2025
@cmbz cmbz added FY25 Sprint 23 FY25 Sprint 23 (2025-05-07 - 2025-05-21) FY25 Sprint 24 FY25 Sprint 24 (2025-05-21 - 2025-06-04) labels May 20, 2025
@poikilotherm poikilotherm self-requested a review June 4, 2025 07:04
@poikilotherm
Copy link
Contributor

@BenediktMeierUIT it seems I cannot push changes to your PR branch. I am aware that you will probably not read this soon, so I am going to close this PR and create a new one, but include your commits. Thanks for your work on this!

@pdurbin
Copy link
Member

pdurbin commented Jun 4, 2025

Replaced by this PR:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Component: Containers Anything related to cloudy Dataverse, shipped in containers. FY25 Sprint 23 FY25 Sprint 23 (2025-05-07 - 2025-05-21) FY25 Sprint 24 FY25 Sprint 24 (2025-05-21 - 2025-06-04) Size: 3 A percentage of a sprint. 2.1 hours.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Dead Dataverse after server restart

3 participants